Skip to content

Add ADR for removing forms-api app#225

Merged
lfdebrux merged 2 commits intomainfrom
ldeb-add-adr-remove-forms-api
Aug 28, 2025
Merged

Add ADR for removing forms-api app#225
lfdebrux merged 2 commits intomainfrom
ldeb-add-adr-remove-forms-api

Conversation

@lfdebrux
Copy link
Copy Markdown
Contributor

@lfdebrux lfdebrux commented Aug 19, 2025

PR Checklist

  • If you are proposing a new decision record document, used the right template for that
    - (ADR, decision-record, engagement, research)
  • Set yourself as the Assignee
  • Tag anyone you would like to review, or @forms-design or @forms-devs
  • Fill in the template below

What

Add an Architecture Decision Record recording our decision to remove the forms-api app from our architecture.

How to review

Rendered version: https://github.com/alphagov/forms/blob/ldeb-add-adr-remove-forms-api/ADR/ADRXXX-remove-forms-api.md

  1. Semantic: Do you agree with the changes?
  2. Syntactic: Spelling, grammar, etc.

Who can review

Any developers on GOV.UK Forms

@lfdebrux lfdebrux self-assigned this Aug 19, 2025
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
@lfdebrux lfdebrux requested a review from a team August 19, 2025 12:44
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
Comment thread ADR/ADRXXX-remove-forms-api.md Outdated
@lfdebrux lfdebrux force-pushed the ldeb-add-adr-remove-forms-api branch 2 times, most recently from 2c21ead to 2d765db Compare August 20, 2025 10:37
@lfdebrux lfdebrux marked this pull request as ready for review August 20, 2025 11:55
@lfdebrux lfdebrux force-pushed the ldeb-add-adr-remove-forms-api branch 2 times, most recently from 1d6bb5b to 8957009 Compare August 21, 2025 07:46
theseanything
theseanything previously approved these changes Aug 27, 2025
Comment thread ADR/ADR045-remove-forms-api.md Outdated
@lfdebrux lfdebrux merged commit 9eb96e4 into main Aug 28, 2025
@lfdebrux lfdebrux deleted the ldeb-add-adr-remove-forms-api branch August 28, 2025 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants